Skip to content

Conversation

@jaclync
Copy link
Contributor

@jaclync jaclync commented Dec 14, 2022

Related to #8118

Description

Testing instructions

Right now, after the checkout step, we wait for the newly created site to become a Jetpack site with 10 retries every 5 seconds. However, when the site still hasn't become a Jetpack site after 10 retries, the app currently just hangs on the installation screen and there's nothing to do except for relaunching the app. In this PR, at the end of the retries, if the site is still not a Jetpack site the store creation flow is dismissed and an alert is presented. The alert informs the merchant that the new site should appear in the store picker shortly and they can contact support if any issues.

Screenshots

simulator.mov

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@jaclync jaclync added the type: bug A confirmed bug. label Dec 14, 2022
@jaclync jaclync added this to the 11.7 milestone Dec 14, 2022
@wpmobilebot
Copy link
Collaborator

You can test the changes from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr8406-623c015 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@selanthiraiyan selanthiraiyan self-assigned this Dec 15, 2022
Copy link
Contributor

@selanthiraiyan selanthiraiyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this by turning off the internet, and Ii works as expected.

LGTM 🚢

@spencertransier spencertransier merged commit 1632c32 into trunk Dec 16, 2022
@spencertransier spencertransier deleted the feat/waiting-for-jetpack-timeout branch December 16, 2022 17:28
@jaclync jaclync changed the title Store creation enhancement: show an alert when the it times out waiting for Jetpack site Store creation enhancement: show an alert when it times out waiting for Jetpack site Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A confirmed bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants